Skip to content

Migrate the EN card 025 to Mermaid - #493

Open
Shivansh-22866 wants to merge 3 commits into
jtemporal:mainfrom
Shivansh-22866:main
Open

Migrate the EN card 025 to Mermaid#493
Shivansh-22866 wants to merge 3 commits into
jtemporal:mainfrom
Shivansh-22866:main

Conversation

@Shivansh-22866

Copy link
Copy Markdown
Contributor

Fixes #139

The EN card 025 has been configured as per the Mermaid configuration

Copilot AI lite review requested due to automatic review settings August 21, 2026 18:04
@netlify

netlify Bot commented Aug 21, 2026

Copy link
Copy Markdown

Deploy Preview for gitfichas ready!

Name Link
🔨 Latest commit a2d68be
🔍 Latest deploy log https://app.netlify.com/projects/gitfichas/deploys/6a8894ba699f690008d62acf
😎 Deploy Preview https://deploy-preview-493--gitfichas.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The post currently contains invalid YAML front matter and is missing the required generated Mermaid SVG asset expected when use_static_image: true is set.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Migrates EN card #25 from the legacy image-based format to the Mermaid-based post template, aligning with the project’s CONTRIBUTING.md migration workflow for Mermaid cards.

Changes:

  • Replaced title/image-based front matter with Mermaid card front matter (pretitle, title, subtitle, command, descriptors, etc.).
  • Removed the legacy markdown body (image + table) and switched to {% include mermaid-graphs.html %} rendering.

Please stage the changes and commit both the post changes AND the generated SVG files together (the Mermaid migration workflow expects the corresponding SVG under assets/img/mermaid/ to be committed with the post).

File summaries
File Description
en/_posts/2021-10-25-025.md Converts EN card 025 front matter/body to the Mermaid post format and switches rendering to the Mermaid include.
Review details

Suppressed comments (1)

en/_posts/2021-10-25-025.md:14

  • use_static_image: true is set, but there is no generated SVG for this card (assets/img/mermaid/025-en.svg) in the repo. Please run python3 scripts/generate_images_only.py "2021-10-25-025.md" (or regenerate with --force) and commit the generated SVG alongside the post.
use_static_image: trueuse_static_image: true
  • Files reviewed: 1/1 changed files
  • Comments generated: 3
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread en/_posts/2021-10-25-025.md Outdated
author: "@jtemporal"
number: "025"
mermaid: true
use_static_image: trueuse_static_image: true
nt:
url: "/en/026"
title: "#026 git branch -D nome"
title: "#026 git branch -d nome"
Comment thread en/_posts/2021-10-25-025.md Outdated
Shivansh-22866 and others added 2 commits August 21, 2026 23:37
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Migrate en 🇺🇸 card 25 to Mermaid

2 participants